home *** CD-ROM | disk | FTP | other *** search
/ Workbench Add-On / Workbench Add-On - Volume 1.iso / Music / MISC / mpegaudio / fixes.txt < prev    next >
Internet Message Format  |  1995-08-23  |  11KB
  1. From:    US1RMC::"seymour@m31.dgbt.doc.ca" "Seymour Shlien" 27-AUG-1993 15:05:31.82
  2. To:    3d::pan
  3. CC:    
  4. Subj:    problems and fixes
  5.  
  6. Dear Davis,
  7.  
  8. I have now got the MPEG audio coder and decoder to
  9. work on both UNIX (Sparc 2) and Microsoft C. All the problems
  10. that we encountered were mainly restricted to the code in the
  11. file common.c and were related to the AIFF
  12. audio format functions. I gather from the comments in
  13. the text, other people had encountered similar problems.
  14.  
  15. When compiled under UNIX,  the gcc compiler
  16. gave messages similar to below.  
  17.  
  18. common.c: In function `aiff_read_headers':
  19. common.c:673: warning: multi-character character constant
  20. common.c:674: warning: multi-character character constant
  21. common.c:688: warning: multi-character character constant
  22. common.c:719: warning: multi-character character constant
  23. common.c:719: warning: passing arg 2 of `strcmp' makes pointer from integer without a cast
  24. common.c: In function `aiff_write_headers':
  25. common.c:807: warning: multi-character character constant
  26. common.c:808: warning: multi-character character constant
  27. common.c:809: warning: multi-character character constant
  28.  
  29.    670     if (fread(&FormChunk, sizeof(Chunk), 1, file_ptr) != 1)
  30.    671        return(-1);
  31.    672   
  32.    673     if (*(unsigned long *) FormChunk.ckID != IFF_ID_FORM ||
  33.    674         *(unsigned long *) FormChunk.formType != IFF_ID_AIFF)
  34.    675        return(-1);
  35.    676
  36.    677
  37.    678  /*   if (strcmp(FormChunk.ckID,IFF_ID_FORM) ||
  38.    679         strcmp(FormChunk.formType,IFF_ID_AIFF))
  39.    680        return(-1);
  40.    681  */
  41.    ...
  42.    ...
  43.    715           aiff_ptr->numSampleFrames = CommChunk.numSampleFrames;
  44.    716           aiff_ptr->sampleSize = CommChunk.sampleSize;
  45.    717   
  46.    718  /*    } else if (*(unsigned long *)Header.ckID == IFF_ID_SSND) { */
  47.    719        } else if (strcmp(Header.ckID,IFF_ID_SSND)) {
  48.    720   
  49.    721           /*
  50.  
  51.  
  52. Similarly on MSDOS we obtained the following warnings
  53.  
  54. Microsoft (R) Program Maintenance Utility   Version 1.11    
  55. Copyright (c) Microsoft Corp 1988-90. All rights reserved.
  56.  
  57.     cl /AL /G2 /FPi87 /nologo /c /Gi musicin.c
  58. musicin.c
  59. musicin.c(959) : warning C4047: '!=' : different levels of indirection
  60.     cl /AL /G2 /FPi87 /nologo /c /Gi common.c
  61. common.c
  62. common.c(814) : warning C4047: '=' : different levels of indirection
  63. common.c(815) : warning C4047: '=' : different levels of indirection
  64. common.c(816) : warning C4047: '=' : different levels of indirection
  65.  
  66.    814   
  67.    815     *(unsigned long *) FormChunk.ckID     = IFF_ID_FORM;
  68.    816     *(unsigned long *) FormChunk.formType = IFF_ID_AIFF;
  69.    817     *(unsigned long *) CommChunk.ckID     = IFF_ID_COMM;
  70.    818
  71.    819     double_to_extended(&aiff_ptr->sampleRate, temp_sampleRate);
  72.   
  73. also a few similar error messages in musicin.c and musicout.c
  74.  
  75.  
  76. Despite these warnings both the encoder (musicin) and the decoder
  77. musicout appeared to run correctly on the data files 
  78. orig.mpg and sine.dec. However command line input is not
  79. properly implemented on MicroSoft C. The program tries to make
  80. a file orig.mpg.dec which is not legal in MSDOS
  81.  
  82.  
  83.  
  84.          BUGS  IN THE SOFTWARE
  85.  
  86. For MS_DOS it was necessary to modify the
  87. mem_alloc function in order to avoid crashing the computer on startup.
  88. I do not know the reason as I am not an expert in MicroSoft C.
  89.  
  90.  
  91. Also, the software does not properly handle AIFF formatted files.
  92. Using the program musicout I created an AIFF formatted file from
  93. the orig.mpg file. When I ran musicin on the AIFF formatted file on
  94. our SUN work station (ie UNIX environment) , the program crashed.
  95. On the other hand, when the same test was performed on the MSDOS,
  96. the computer did not crash but it failed to recognize the
  97. file as an AIFF formatted file and it did not read the header block.
  98. Nevertheless it was still able to compress the file, once the appropriate
  99. header information was typed in. 
  100.  
  101.  
  102. The problems seem to be related to the handling of the ID information
  103. in the chunk structure of the AIFF file. The ID is a 32 bit integer   
  104. which can take one of 5 designated values. The designated values
  105. for convenience also can be decoded as ascii strings FORM, AIFF
  106. COMM, SSND and MPEG. The ISO software attempts to provide two ways
  107. of handling these numbers- either as a multicharacter constant or
  108. a string.
  109.  
  110. In the common.h file
  111.  
  112. /*
  113.  * Note:  The value of a multi-character constant
  114.  *        is implementation-defined.
  115.  */
  116. #if !defined(MS_DOS) && !defined(AIX)
  117.  
  118. #define         IFF_ID_FORM             'FORM'
  119. #define         IFF_ID_AIFF             'AIFF'
  120. #define         IFF_ID_COMM             'COMM'
  121. #define         IFF_ID_SSND             'SSND'
  122. #define         IFF_ID_MPEG             'MPEG'
  123. #else
  124. #define         IFF_ID_FORM             "FORM"
  125. #define         IFF_ID_AIFF             "AIFF"
  126. #define         IFF_ID_COMM             "COMM"
  127. #define         IFF_ID_SSND             "SSND"
  128. #define         IFF_ID_MPEG             "MPEG"
  129. #endif
  130.  
  131.  
  132.  
  133. Unfortunately, the revisions to the code in common.c
  134. do not properly read or write the AIFF header block in
  135. either mode.
  136.  
  137. If the IFF_* constants are defined as a long integer, (ie
  138. single quotes around FORM, AIFF, etc.), then the strcmp
  139. function will crash on some machines because the inputs
  140. Header.ckID and IFF_ID_FORM are not strings. Strcmp
  141. expects to receive null terminated strings. Since it
  142. instead receives addresses to nonterminated strings, the
  143. strcmp runs off to infinity searching for the null character.
  144.  
  145. Note: when IFF_* are defined as strings, Header.ckID is
  146. still not a null terminated string. Fortunately strcmp
  147. stops when a null is encountered in either one of its 
  148. arguments -- in this case a null is encountered in
  149. IFF_*. It might be safer to use strncmp which includes
  150. a string length argument.
  151.  
  152.   
  153.  
  154. This was the source of our problem on the UNIX machine as
  155. the common.c attempted to do a string compare in one
  156. spot (around line 718). 
  157.  
  158. /*    } else if (*(unsigned long *)Header.ckID == IFF_ID_SSND) { */
  159.       } else if (strcmp(Header.ckID,IFF_ID_SSND)) {
  160.  
  161. The commented line would have worked correctly in the
  162. UNIX environment.
  163.  
  164.  
  165. A different problem occurs in MS_DOS. In the function 
  166. aiff_write_header the following statement does not do what
  167. was intended when IFF_ID_FORM is a string.
  168.  
  169.   *(unsigned long *) FormChunk.ckID     = IFF_ID_FORM;
  170.  
  171. IFF_ID_FORM points to an address of the string "FORM". Though
  172. it was intended to put "FORM" in FormChunk.ckID, the above
  173. statement instead places the address of IFF_ID_FORM into
  174. FormChunk.ckID. This is what is put in the AIFF file. Musicin
  175. reads the file and fails to recognize it as an AIFF file. (The
  176. above statement would have been correct if IFF_ID_FORM was
  177. a long integer.) 
  178.  
  179.  
  180.                FIXES
  181.  
  182. I have made the following changes in the software so that it
  183. can now handle AIFF formatted files in either MicroSoft C or
  184. UNIX.
  185.  
  186. In common.h, I have introduced a new def called IFF_LONG
  187. whenever IFF_* is represented by multicharacter longs instead
  188. of strings. 
  189.  
  190. 187,188c187
  191. < #if !defined(MS_DOS) && !defined(AIX)  
  192. < #define         IFF_LONG
  193. ---
  194. > #if !defined(MS_DOS) && !defined(AIX)
  195.  
  196.  
  197.  
  198.  
  199. In common.c the following changes were made
  200.  
  201.  
  202. For avoiding crashing the PC on startup
  203. 456,457c456
  204. <     /*ptr = (void FAR *) _fmalloc((unsigned int)block);*/ /* far memory, 92-07-08 sr */
  205. <     ptr = (void FAR *) malloc((unsigned int)block); /* far memory, 93-08-24 ss */
  206. ---
  207. >     ptr = (void FAR *) _fmalloc((unsigned int)block); /* far memory, 92-07-08 sr */
  208.  
  209.  
  210.  
  211. For debugging and testing
  212. 646,656d644
  213. < 
  214. < /****  for debugging 
  215. < showchar(str)
  216. < char str[4];
  217. < {
  218. < int i;
  219. < for (i=0;i<4;i++) printf("%c",str[i]);
  220. < printf("\n");
  221. < }
  222. < ****/
  223. < 
  224.  
  225.  
  226. To correct the problems with the IFF_* ID's I do
  227. the string comparisons two different ways depending
  228. on whether IFF_LONG is defined. 
  229.  
  230.  
  231.  
  232. 685d672
  233. < #ifdef IFF_LONG 
  234. 689d675
  235. < #else
  236. 691,694d676
  237. <    if (strncmp(FormChunk.ckID,IFF_ID_FORM,4) ||
  238. <        strncmp(FormChunk.formType,IFF_ID_AIFF,4))
  239. <       return(-1);
  240. < #endif
  241. 695a678,681
  242. > /*   if (strcmp(FormChunk.ckID,IFF_ID_FORM) ||
  243. >        strcmp(FormChunk.formType,IFF_ID_AIFF))
  244. >       return(-1);
  245. > */
  246. 702d687
  247. < #ifdef IFF_LONG  
  248. 705,708c690,691
  249. < #else
  250. <       if (strncmp(Header.ckID,IFF_ID_COMM,4) == 0) {
  251. < #endif
  252. < 
  253. ---
  254. > /*      if (strcmp(Header.ckID,IFF_ID_COMM)) {
  255. > */
  256. 711a695
  257. > 
  258. 734,738c718,720
  259. < #ifdef IFF_LONG 
  260. <       } else if (*(unsigned long *)Header.ckID == IFF_ID_SSND) { 
  261. < #else
  262. <       } else if (strncmp(Header.ckID,IFF_ID_SSND,4) == 0) {
  263. < #endif
  264. ---
  265. > /*    } else if (*(unsigned long *)Header.ckID == IFF_ID_SSND) { */
  266. >       } else if (strcmp(Header.ckID,IFF_ID_SSND)) {
  267. > 
  268. 741a724
  269. > 
  270. 824d806
  271. < #ifdef IFF_LONG 
  272. 828,832d809
  273. < #else
  274. <    strncpy(FormChunk.ckID,IFF_ID_FORM,4);
  275. <    strncpy(FormChunk.formType,IFF_ID_AIFF,4);
  276. <    strncpy(CommChunk.ckID,IFF_ID_COMM,4);
  277. < #endif
  278. 1536a1514
  279. > 
  280.  
  281.  
  282.  
  283.  
  284. In musicin.c
  285.  
  286. The following changes were made for the same reasons discussed above.
  287.  
  288.  
  289. 959d958
  290. < #ifdef IFF_LONG
  291. 961,963d959
  292. < #else
  293. <     if (strncmp(&pcm_aiff_data->sampleType,IFF_ID_SSND,4)) {
  294. < #endif
  295.  
  296.  
  297.  
  298. In musicout.c
  299.  
  300. The following changes were made for the same reasons discussed above.
  301.  
  302.  
  303. 381d380
  304. < #ifdef IFF_LONG       
  305. 383,385d381
  306. < #else
  307. <        strncpy(&pcm_aiff_data.sampleType,IFF_ID_SSND,4);
  308. < #endif
  309. 388c384
  310. <  
  311. ---
  312. > 
  313.  
  314.  
  315. This is a copy of the makefile that I am using on the IBM. Perhaps
  316. one of the switches might explain why I had trouble with the 
  317. mem_alloc.
  318.  
  319. ALL : musicin.exe musicout.exe
  320.  
  321. CFLAGS = /AL /G2 /FPi87 /nologo /c /Gi /F 256 
  322. LFLAGS= /SE:256 /ST:16000 /F /PACKC
  323.  
  324. musicin.obj: musicin.c common.h encoder.h
  325.     cl $(CFLAGS) musicin.c
  326.  
  327. common.obj: common.c common.h
  328.     cl $(CFLAGS) common.c
  329.  
  330. encode.obj: encode.c common.h encoder.h
  331.     cl $(CFLAGS) encode.c
  332.  
  333. subs.obj: subs.c common.h encoder.h
  334.     cl $(CFLAGS) subs.c
  335.  
  336. psy.obj: psy.c common.h encoder.h
  337.     cl $(CFLAGS) psy.c
  338.  
  339. tonal.obj: tonal.c common.h encoder.h
  340.     cl $(CFLAGS) tonal.c
  341.  
  342. musicout.obj: musicout.c common.h decoder.h
  343.     cl $(CFLAGS) musicout.c
  344.  
  345. decode.obj: decode.c common.h decoder.h
  346.     cl $(CFLAGS) decode.c
  347.  
  348. musicin.exe: musicin.obj common.obj encode.obj subs.obj psy.obj tonal.obj
  349.     link $(LFLAGS) musicin common encode subs psy tonal,musicin,,;
  350.  
  351. musicout.exe: musicout.obj common.obj decode.obj subs.obj
  352.     link $(LFLAGS) musicout common decode subs,musicout,,;
  353.  
  354.  
  355.  
  356.  
  357. I am grateful, for the help from Daniel Lauzon -- our resident C and C++
  358. expert. Also Bill Truerniet helped me with Microsoft C 
  359. on the PC.
  360.    
  361.  
  362. seymour@dgbt.doc.ca
  363.  
  364.  
  365. % ====== Internet headers and postmarks (see DECWRL::GATEWAY.DOC) ======
  366. % Received: by us1rmc.bb.dec.com; id AA06920; Fri, 27 Aug 93 14:58:09 -0400
  367. % Received: by inet-gw-1.pa.dec.com; id AA02547; Fri, 27 Aug 93 11:59:18 -0700
  368. % Received: from rigel.dgbt.doc.ca by  m31.dgbt.doc.ca (4.1/SMI-4.1) id AA01692; Fri, 27 Aug 93 14:57:59 ED
  369. % Date: Fri, 27 Aug 93 14:57:59 EDT
  370. % From: seymour@m31.dgbt.doc.ca (Seymour Shlien)
  371. % Message-Id: <9308271857.AA01692@ m31.dgbt.doc.ca>
  372. % To: 3d::pan
  373. % Subject: problems and fixes
  374.